-
Notifications
You must be signed in to change notification settings - Fork 0
Feature/SOF-7776 WFN #98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
| x0(1)=0.0, x0(2)=0.0, x0(3)=0.0, ! line origin (alat units) | ||
| e1(1)=0.0, e1(2)=0.0, e1(3)=1.0, ! direction along z (alat units) | ||
| nx = 200 ! samples along the line | ||
| / |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check formatting for other pp files and match
| # ---------------------------------------------------------------- # | ||
| import json | ||
|
|
||
| from munch import Munch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's explain why this is needed and put a task to remove the use of it
| flavorName: plot_wavefunction | ||
| name: plot WFN | ||
| results: | ||
| - name: potential_profile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
|
|
||
| # Create potential_profile JSON for platform rendering | ||
| wavefunction_data = { | ||
| "name": "potential_profile", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wavefunction_amplitude
| results: | ||
| - file_content | ||
| - 'workflow:pyml_predict' | ||
| - potential_profile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
| execName: python | ||
| flavorName: extract_bands_fermi | ||
| name: extract_bands_fermi | ||
| flowchartId: extract-band-fermi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
extract-bands-fermi? here and below
| prefix = '__prefix__' | ||
| plot_num = 7 | ||
| kpoint = 1 | ||
| kband = {% raw %}{{ KBAND_VALUE }}{% endraw %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can set default value to 1
| } | ||
|
|
||
| # Print to STDOUT for subsequent assignment unit | ||
| print(json.dumps(result, indent=4)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not be template, it can be an assignment unit or split into multiple assignment units
| } | ||
|
|
||
| # Print JSON to STDOUT (will be captured as potential_profile result) | ||
| print(json.dumps(wavefunction_data, indent=2)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove 33-49 - it should be defined as a separate property
| # # | ||
| # ------------------------------------------------------------------ # | ||
|
|
||
| munch==2.5.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove
No description provided.